Skip to content

Partition ocean freshwater: rain through ice, snow to open water#165

Open
simone-silvestri wants to merge 4 commits intoss/snow-model-integrationfrom
ss/ocean-freshwater-partitioning
Open

Partition ocean freshwater: rain through ice, snow to open water#165
simone-silvestri wants to merge 4 commits intoss/snow-model-integrationfrom
ss/ocean-freshwater-partitioning

Conversation

@simone-silvestri
Copy link
Copy Markdown
Member

Summary

Split the ocean freshwater budget:

  • Rain reaches the ocean everywhere (runs through cracks in ice)
  • Snow only reaches the ocean through the open-water fraction (1-ℵ);
    snow on ice is routed to the sea ice model as snowfall
  • Evaporation is from the open-water fraction (1-ℵ)

The total freshwater flux ΣFao now has ice-partitioning built in,
so Jˢao = -S × ΣFao no longer needs additional (1-ℵ) scaling.

Depends on #164

Test plan

  • Verify freshwater budget closes (rain + snow + evap + ice-ocean = total)
  • Check salinity tendencies have correct sign for rain/evap/melt

@simone-silvestri simone-silvestri force-pushed the ss/snow-model-integration branch from 963ba94 to 8b6e085 Compare April 15, 2026 10:56
@simone-silvestri simone-silvestri force-pushed the ss/ocean-freshwater-partitioning branch 2 times, most recently from daadd7e to 2e69440 Compare April 15, 2026 11:58
@simone-silvestri simone-silvestri force-pushed the ss/snow-model-integration branch from 0d8881e to 8e3cb6b Compare April 15, 2026 12:31
Split the ocean freshwater budget so that:
- Rain reaches the ocean everywhere (runs through cracks in ice)
- Snow only reaches the ocean through the open-water fraction (1-ℵ);
  snow on ice is routed to the sea ice model as snowfall
- Evaporation is from the open-water fraction (1-ℵ)

The total freshwater flux ΣFao now has ice-partitioning built in,
so the salinity flux Jˢao = -S × ΣFao no longer needs an additional
(1-ℵ) scaling.

Requires the Jˢⁿ (interpolated snowfall) field from PR #4.
@simone-silvestri simone-silvestri force-pushed the ss/ocean-freshwater-partitioning branch from 2e69440 to 2d3e239 Compare April 15, 2026 12:31
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/Oceans/assemble_net_ocean_fluxes.jl 0.00% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant